Uploaded image for project: 'Calcite'
  1. Calcite
  2. CALCITE-3851

Replace the node importance map with a set for pruned nodes

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Closed
    • Minor
    • Resolution: Fixed
    • None
    • 1.23.0
    • core

    Description

      Currently, volcano planner stores rel node importances with a map. In fact, the value of the map can only be 0.
      So there is no need to store the values, and we replace the map with a set to store nodes whose importances are 0.

      This makes the code logic clearer, and avoids instability caused by comparing floating point values.

      Attachments

        Issue Links

          Activity

            People

              Chunwei Lei Chunwei Lei
              fan_li_ya Liya Fan
              Votes:
              0 Vote for this issue
              Watchers:
              6 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Time Tracking

                  Estimated:
                  Original Estimate - Not Specified
                  Not Specified
                  Remaining:
                  Remaining Estimate - 0h
                  0h
                  Logged:
                  Time Spent - 3h 50m
                  3h 50m